Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[js/webgpu] fix buffer size when download #15990

Merged
merged 1 commit into from
May 20, 2023

Conversation

fs-eire
Copy link
Contributor

@fs-eire fs-eire commented May 17, 2023

Description

fix buffer size when download. buffer size should always be padded to multiple of 4.

resolved issue described in #15796

Image

@fs-eire
Copy link
Contributor Author

fs-eire commented May 17, 2023

@xenova you may want to try if this fixes your error.

@xenova
Copy link

xenova commented May 18, 2023

@fs-eire
Copy link
Contributor Author

fs-eire commented May 18, 2023

you need to select the branch filter. the artifacts for this PR is here: https://dev.azure.com/onnxruntime/onnxruntime/_build/results?buildId=1010206&view=results

@xenova
Copy link

xenova commented May 18, 2023

Thanks - got that working. New issue though:
image

@fs-eire
Copy link
Contributor Author

fs-eire commented May 19, 2023

Thanks - got that working. New issue though: image

will do a separated PR for this.

@fs-eire fs-eire merged commit 18f17c5 into main May 20, 2023
@fs-eire fs-eire deleted the fs-eire/webgpu-fix-buffer-size-download branch May 20, 2023 07:21
fs-eire added a commit that referenced this pull request May 30, 2023
### Description
this is a following up fix for #15990, which should resolve the
RangeError issue.
siweic0 pushed a commit to siweic0/onnxruntime-web that referenced this pull request May 9, 2024
### Description
fix buffer size when download. buffer size should always be padded to
multiple of 4.

resolved issue described in microsoft#15796

>
![Image](https://user-images.githubusercontent.com/26504141/239093785-9417dffc-6f00-47b2-956d-402b43bdb0a9.png)
siweic0 pushed a commit to siweic0/onnxruntime-web that referenced this pull request May 9, 2024
### Description
this is a following up fix for microsoft#15990, which should resolve the
RangeError issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants